Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: deal scraper RFC #631

Closed
wants to merge 2 commits into from
Closed

docs: deal scraper RFC #631

wants to merge 2 commits into from

Conversation

aidan46
Copy link
Contributor

@aidan46 aidan46 commented Dec 10, 2024

Description

This PR adds an RFC for the on-chain deal information scraper, which maps Peer IDs to piece_cid's. The application will need to maintain a database that holds these mapping, the database is not mentioned in this RFC but should be considered when implementing. This is an initial draft, I will do some testing to check if this will work.

@aidan46 aidan46 self-assigned this Dec 10, 2024
@jmg-duarte jmg-duarte added the rfc Request for Comments (RFC) label Dec 10, 2024
@jmg-duarte jmg-duarte changed the title docs: Add deal scraper RFC docs: deal scraper RFC Dec 10, 2024

## Abstract

This RFC proposes the design and functionality of an application to map `piece_cids` to their respective owners, identified by [Peer ID][1]s, in the Polka-Storage ecosystem.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm wondering, who needs that from the business perspective at this point.
When a Storage Client is making a deal, they know with whom, so they should know where their file is stored.
The only thing they need is to resolve PeerID to a Multiaddress and then connect to the SP, download the file and that's it.

Not sure who the user of it would be.

@aidan46 aidan46 closed this Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rfc Request for Comments (RFC)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants